Update Orleans and Aspire integration documentation - #54201
Conversation
.NET Aspire -> Aspire
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the Orleans hosting documentation to reflect “Aspire” (vs “.NET Aspire”) terminology and refreshes related setup links.
Changes:
- Renamed headings/front-matter to “Orleans and Aspire integration”.
- Updated external/reference links (aspire.dev, VS Code extension link).
- Adjusted wording in pivoted sections to use “Aspire” terminology.
dotnet-policy-service
Bot
added
the
community-contribution
|
@Meir017, I agree with the Copilot suggestions. Here are the issues and fixes: Issue 1: Branding inconsistency
Fix: Keep ".NET Aspire" as the primary name throughout the document. If shorthand is needed, add "(Aspire)" after first mention, e.g., ".NET Aspire (hereafter referred to as Aspire)". Issue 2: External links vs internal links Fix: Restore the Microsoft Learn links for consistency, localization, and long-term stability. Use aspire.dev only for content that doesn't exist on Microsoft Learn. Issue 3: Capitalization Issue 4: Mixed terminology in pivot section Summary of changes needed: | Title | "Orleans and Aspire integration" | "Orleans and .NET Aspire integration" | I can help create a PR with these fixes if needed. |
|
The official name is Aspire, not .NET Aspire |
|
@copilot resolve the merge conflicts in this pull request |
Co-authored-by: Genevieve Warren <[email protected]>
.NET Aspire -> Aspire
Summary
FYI @IEvangelist @davidfowl